Skip to content

chore(ci): fix failure in detecting compilers in clang-tidy job - #4642

Merged
lidavidm merged 1 commit into
apache:mainfrom
amoeba:fix/clang-tidy-compiler-path
Aug 2, 2026
Merged

chore(ci): fix failure in detecting compilers in clang-tidy job#4642
lidavidm merged 1 commit into
apache:mainfrom
amoeba:fix/clang-tidy-compiler-path

Conversation

@amoeba

@amoeba amoeba commented Jul 31, 2026

Copy link
Copy Markdown
Member

The clang-tidy job recently started failing.

Last success: https://github.com/apache/arrow-adbc/actions/runs/30502165159/job/90743952843
First failure: https://github.com/apache/arrow-adbc/actions/runs/30596999777/job/91051511466

The error is,

Could not find the compiler specified in the environment variable CC: /home/runner/work/arrow-adbc/arrow-adbc/build/tidy/x86_64-conda-linux-gnu-cc.

Note that path to x86_64-conda-linux-gnu-cc is relative to the CWD which is wrong.

The difference seems to be:

In a successful run, I see:

++ readlink -f /home/runner/miniconda3/envs/test/bin/x86_64-conda-linux-gnu-cc
++ readlink -f /home/runner/miniconda3/envs/test/bin/x86_64-conda-linux-gnu-c++

In a failed run, I see:

++ readlink -f x86_64-conda-linux-gnu-cc
++ readlink -f x86_64-conda-linux-gnu-c++

A notable difference between these two environments is that the last working build used gcc_linux-64=14.3.0 whereas the failures started after gcc_linux-64=14.4.0 showed up.

@amoeba amoeba changed the title chore(ci): chore(ci): fix failure in detecting compilers in clang-tidy job Jul 31, 2026
@amoeba

amoeba commented Jul 31, 2026

Copy link
Copy Markdown
Member Author

Seems plausible it was caused by https://github.com/conda-forge/ctng-compiler-activation-feedstock/blob/main/recipe/build_scripts.sh#L222-L224 which looks like it just came out.

I can reproduce what see in CI with:

$ conda create -n test-old gcc_linux-64=14.3.0 -c conda-forge
$ conda activate test-old
$ echo $CC
/opt/conda/envs/test-old/bin/x86_64-conda-linux-gnu-cc
$ conda create -n test-new gcc_linux-64=14.4.0 -c conda-forge
$ conda activate test-new
$ echo $CC
x86_64-conda-linux-gnu-cc

@amoeba
amoeba marked this pull request as ready for review July 31, 2026 22:53
@amoeba
amoeba requested review from kou and lidavidm as code owners July 31, 2026 22:53
@lidavidm
lidavidm merged commit 42d1628 into apache:main Aug 2, 2026
83 of 87 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants